[CI] Add Windows Blender test to nightly by sarnex · Pull Request #21718 · intel/llvm

sarnex · 2026-04-09T13:48:38Z

This PR introduces a real world test into CI.

The idea is we build and run Blender using the just-built SYCL compiler (initially, only the nightly) and then run an example workflow on a (multi) GPU system.

The first version of this PR only tracks pass/fail, but in the future we could add a performance threshold to cause a fail.
First we build OIDN and Embree, both of which use SYCL and DPC++. Then we build Blender with the just-built OIDN, Embree and SYCL compiler.

The steps of this workflow (as in, what to do at a high level) were developed with help from internal Blender experts.

Run in nightly example: https://github.com/intel/llvm/actions/runs/24317863959/job/70999916822

Entire workflow takes ~10 minutes.

sarnex · 2026-04-09T14:50:51Z

devops/actions/blender/blender-build/action.yml

+      run: |
+        echo "SCCACHE_DIR=$env:CCACHE_DIR_INPUT" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+        echo "SCCACHE_MAXSIZE=10G" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+        # Wthout this we get a linking error about not being able to find sycl-devicelib-host.lib
+        echo "LIB=$env:LIB;$env:SYCL_DIR_INPUT\lib" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+        echo "CMAKE_EXTRA_ARGS=-DWITH_WINDOWS_SCCACHE=ON" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append


@KornevNikita Do we need these checks to pass before merging PRs or are we allowed to ignore them?

This warning seems to suggest we aren't allowed to modify any environment variables with non-static strings, and we have to set these vars to get things to work. We already do that heavily in other workflows, it's just the checks were added after those workflows were written. Thanks

Yeah, this is annoying, I suggest we ignore these :)

I like that answer, thanks :P

devops/actions/blender/blender-build/action.yml

.github/workflows/sycl-blender-build-and-test.yml

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

KornevNikita · 2026-04-13T15:19:40Z

do we need to clean up anything afterwards?

KornevNikita · 2026-04-13T15:20:20Z

.github/workflows/sycl-blender-build-and-test.yml

+    inputs:
+      runner:
+        type: string
+        required: True


Suggested change

required: True

required: true

KornevNikita · 2026-04-13T15:20:46Z

devops/actions/blender/blender-build/action.yml

+      run: |
+        echo "SCCACHE_DIR=$env:CCACHE_DIR_INPUT" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+        echo "SCCACHE_MAXSIZE=10G" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+        # Wthout this we get a linking error about not being able to find sycl-devicelib-host.lib


Suggested change

# Wthout this we get a linking error about not being able to find sycl-devicelib-host.lib

# Without this we get a linking error about not being able to find sycl-devicelib-host.lib

sarnex · 2026-04-13T15:22:03Z

On the next run in the checkout phase, it runs git clean on the entire directory, so it gets cleaned then. Example from the logs:

Cleaning the repository
  "C:\Program Files\Git\cmd\git.exe" clean -ffdx
  Removing blender-build/
  Removing blender-install/
  Removing blender-src/
  Removing embree-build/
  Removing embree-install/
  Removing embree-src/
  Removing install/
  Removing oidn-build/
  Removing oidn-install/
  Removing oidn-src/
  "C:\Program Files\Git\cmd\git.exe" reset --hard HEAD
  HEAD is now at d8e53e1 [CI] Add Windows Blender test to nightly

KornevNikita · 2026-04-13T15:37:16Z

On the next run in the checkout phase, it runs git clean on the entire directory, so it gets cleaned then. Example from the logs:

Cleaning the repository
  "C:\Program Files\Git\cmd\git.exe" clean -ffdx
  Removing blender-build/
  Removing blender-install/
  Removing blender-src/
  Removing embree-build/
  Removing embree-install/
  Removing embree-src/
  Removing install/
  Removing oidn-build/
  Removing oidn-install/
  Removing oidn-src/
  "C:\Program Files\Git\cmd\git.exe" reset --hard HEAD
  HEAD is now at d8e53e1 [CI] Add Windows Blender test to nightly

Shouldn't we manually clean up the space like we do it for E2E/SYCL-CTS tests?

sarnex · 2026-04-13T15:38:48Z

Shouldn't we manually clean up the space like we do it for E2E/SYCL-CTS tests?

Yeah that's cleaner, let me do that, thanks

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

sarnex · 2026-04-13T18:00:17Z

Feedback should be addressed in latest commit, thanks

Test CI run here https://github.com/intel/llvm/actions/runs/24355748290/job/71130372156

.github/workflows/sycl-blender-build-and-test.yml

devops/actions/blender/blender-build/action.yml

devops/actions/blender/embree/action.yml

devops/actions/blender/oidn/action.yml

KornevNikita · 2026-04-13T18:13:38Z

Feedback should be addressed in latest commit, thanks

Test CI run here https://github.com/intel/llvm/actions/runs/24355748290/job/71130372156

Thanks!

KornevNikita · 2026-04-14T10:43:18Z

devops/actions/blender/blender-build/action.yml

+runs:
+  using: "composite"
+  steps:
+    - uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756


Suggested change

- uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756

- uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0

Same below and below and below. Not a blocker, feel free to ignore/make a follow up. I'm updating "uses: ..." across whole repo, so it'd be nice to have this.

I'll add this then merge, thanks

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

github-advanced-security AI found potential problems Apr 9, 2026

View reviewed changes

sarnex force-pushed the sarnex/blender branch 2 times, most recently from 04d162e to d8e53e1 Compare April 10, 2026 18:34

[CI] Add Windows Blender test to nightly

2f60a15

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

sarnex force-pushed the sarnex/blender branch from d8e53e1 to 2f60a15 Compare April 12, 2026 22:24

sarnex marked this pull request as ready for review April 13, 2026 14:38

sarnex requested a review from a team as a code owner April 13, 2026 14:38

sarnex requested review from KornevNikita and uditagarwal97 April 13, 2026 14:38

KornevNikita approved these changes Apr 13, 2026

View reviewed changes

KornevNikita reviewed Apr 13, 2026

View reviewed changes

Address feedback

28fb1c2

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

github-advanced-security AI found potential problems Apr 13, 2026

View reviewed changes

KornevNikita approved these changes Apr 13, 2026

View reviewed changes

KornevNikita reviewed Apr 14, 2026

View reviewed changes

sarnex mentioned this pull request Apr 14, 2026

[CI] Make dependabot upd composite actions #21761

Merged

add action version comment

9756efd

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

sarnex merged commit f390758 into sycl Apr 14, 2026
14 of 15 checks passed

sarnex deleted the sarnex/blender branch April 14, 2026 14:31

	# Wthout this we get a linking error about not being able to find sycl-devicelib-host.lib
	# Without this we get a linking error about not being able to find sycl-devicelib-host.lib

	- uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756
	- uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0

Conversation

sarnex commented Apr 9, 2026 • edited by KornevNikita Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sarnex Apr 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

KornevNikita Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

sarnex Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KornevNikita commented Apr 13, 2026

Uh oh!

KornevNikita Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

KornevNikita Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

sarnex commented Apr 13, 2026

Uh oh!

KornevNikita commented Apr 13, 2026

Uh oh!

sarnex commented Apr 13, 2026

Uh oh!

sarnex commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KornevNikita commented Apr 13, 2026

Uh oh!

KornevNikita Apr 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sarnex Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

sarnex commented Apr 9, 2026 •

edited by KornevNikita

Loading

sarnex Apr 9, 2026 •

edited

Loading

sarnex commented Apr 13, 2026 •

edited

Loading

KornevNikita Apr 14, 2026 •

edited

Loading